Skip to content

build(deps): upgrade project to support pipfile >= 2 #293

Open
lf-novelt wants to merge 4 commits intoemnify:mainfrom
lf-novelt:main
Open

build(deps): upgrade project to support pipfile >= 2 #293
lf-novelt wants to merge 4 commits intoemnify:mainfrom
lf-novelt:main

Conversation

@lf-novelt
Copy link

@lf-novelt lf-novelt commented Feb 19, 2026

I need to integrate this SDK with FastAPI 0.128 which requires pydantic >= 2

this PR used https://github.com/pydantic/bump-pydantic to perform the migration from v1 to v2
image

I saw #268 but it's probably more future proof to upgrade at some point

@lf-novelt lf-novelt changed the title DEPS: upgrade project to support pipfile >= 2 build(deps): upgrade project to support pipfile >= 2 Feb 19, 2026
@marwin-roth
Copy link
Contributor

Hey! Sorry to keep you waiting so long. I fixed the local tests because there were some validation errors. But it seems like the tests in CI are not working properly atm. I have to take a look at that and would also need to test it e2e. Does this requirement still exist for you or not? If so, I can continue with it, if not, I would close the pr.

@lf-novelt
Copy link
Author

lf-novelt commented Mar 10, 2026

hello, thanks for replying.
yes I still need pydantic >= 2, but I can work with my local fork in the meanwhile.
I guess it won't be long until someone else also needs it. as pydantic v1 will be less and less supported by other libs, so this SDK will become harder to integrate as time goes.
Up to you if worth upgrading I guess :)

@marwin-roth
Copy link
Contributor

Ok great that you have a workaround! We will prioritize this later then, and will keep you in the loop. Thanks for your effort so far!

@lf-novelt
Copy link
Author

lf-novelt commented Mar 10, 2026

ok let me check if can make the workflow pass just by updating the PipFile.lock

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants